Skip to content

PM DEVICE RUNTIME: Ifdef out unused events if PM_DEVICE_RUNTIME_ASYNC=n #93727

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bjarki-andreasen
Copy link
Contributor

The struct k_event event is only used if CONFIG_PM_DEVICE_RUNTIME_ASYNC is selected, but EVENTS is selected, and the struct k_event included in the struct pm_device_base if CONFIG_PM_DEVICE_RUNTIME is selected.

Correct to only include event and EVENTS if CONFIG_PM_DEVICE_RUNTIME_ASYNC is selected.

@bjarki-andreasen bjarki-andreasen changed the title PM DEVICE RUNTIME: Only select EVENTS if PM_DEVICE_RUNTIME_ASYNC)Y PM DEVICE RUNTIME: Only select EVENTS if PM_DEVICE_RUNTIME_ASYNC=y Jul 25, 2025
@bjarki-andreasen bjarki-andreasen changed the title PM DEVICE RUNTIME: Only select EVENTS if PM_DEVICE_RUNTIME_ASYNC=y PM DEVICE RUNTIME: Ifdef out unused events if PM_DEVICE_RUNTIME_ASYNC=n Jul 28, 2025
The k_event event is only used if CONFIG_PM_DEVICE_RUNTIME_ASYNC is
selected, but EVENTS is selected, and the k_event included in the
struct pm_device if CONFIG_PM_DEVICE_RUNTIME is selected.

Correct to only include event and EVENTS if
CONFIG_PM_DEVICE_RUNTIME_ASYNC is selected. This saves us 1
struct event per PM DEVICE.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
@bjarki-andreasen bjarki-andreasen force-pushed the pm-device-runtime-fix-event-select branch from 836e6ef to 24445eb Compare July 28, 2025 09:53
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants